Skip to content

[17.0][MIG] l10n_br_account #3757

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1,549 commits into
base: 17.0
Choose a base branch
from

Conversation

marcelsavegnago
Copy link
Member

No description provided.

weblate and others added 30 commits April 28, 2025 16:43
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Currently translated at 97.9% (141 of 144 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
use the native Odoo account invoice tests to ensure
l10n_br_account don't destroy native accounting features
for non Brazilian companies.
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch 2 times, most recently from 2f9695f to c023f8a Compare April 30, 2025 16:07
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch from c023f8a to 288ba2c Compare April 30, 2025 16:13
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch from 51da302 to d5e9cae Compare April 30, 2025 17:52
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch 2 times, most recently from 2caa270 to 532dc6a Compare April 30, 2025 21:01
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch from 11434b8 to 28c0201 Compare April 30, 2025 21:18
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch 3 times, most recently from 05bd839 to bd084f2 Compare May 7, 2025 01:52
@marcelsavegnago
Copy link
Member Author

marcelsavegnago commented May 14, 2025

Peguei uma questão durante os testes.

No modulo l10n_br_base os dados da empresa principal é alterado inclusive mudando a moeda para BRL. Porém, na versão 17 tem uma linha no metodo setupclass do arquivo common.py (pasta tests) que força a mudança da moeda para USD e no caso dá erro porque esta empresa já tem lançamentos de diario neste momento.

image

https://github.com/odoo/odoo/blob/05734a7ae5f6c6ec272178a1ab364db3c88af64d/addons/account/tests/common.py#L50

odoo/odoo@b1537a0

cc @rvalyi @antoniospneto @kaynnan @mileo

@antoniospneto
Copy link
Contributor

@marcelsavegnago talvez uma alternativa é a gente usar as empresas simples nacional e lucro presumido para os testes e demonstrações da nossa localização, e deixar de usar a empresa principal.

@rvalyi
Copy link
Member

rvalyi commented May 14, 2025

@marcelsavegnago talvez uma alternativa é a gente usar as empresas simples nacional e lucro presumido para os testes e demonstrações da nossa localização, e deixar de usar a empresa principal.

Seria bem razoável sim. Mas ai seria interessante começar a fazer isso na 16.0 para não começar a divergir muito nos testes, senão vai ser foda manter viu... Pros testes que não mexem com valores como CRM, hr, zip ou coisa do tipo ta de boa continuar a usar o main_company.

@rvalyi
Copy link
Member

rvalyi commented May 14, 2025

@marcelsavegnago @antoniospneto outra saida que pode ser mais simples a curto prazo seria habilitar de fazer o write de volta para a moeda base.USD aqui no res_company onde ja fizemos uma gambiarra pro base.BRL:

https://github.com/OCA/l10n-brazil/blob/16.0/l10n_br_base/models/res_company.py#L168

Mas assim a longo prazo vai ser melhor focar os testes nas empresas simples e lucro presumido sim.

@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch from bd084f2 to c157d3f Compare May 19, 2025 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.